Skip to content

feat(ci): package-registry provenance#38

Merged
hyperpolymath merged 1 commit into
mainfrom
feat/package-provenance
Jun 25, 2026
Merged

feat(ci): package-registry provenance#38
hyperpolymath merged 1 commit into
mainfrom
feat/package-provenance

Conversation

@hyperpolymath

Copy link
Copy Markdown
Owner

Package-registry provenance

Adds native build provenance attestation to the npm publish workflow (.github/workflows/publish.yml).

Per-ecosystem changes

  • npm (npmjs.org): added job-level permissions: block (contents: read + id-token: write) to the publish-npm job, and changed npm publish --access public -> npm publish --provenance --access public. The job publishes to the public registry (registry-url: https://registry.npmjs.org), so provenance applies.

Notes

  • This repo has no package.json (it is a Deno/ReScript project per CLAUDE.md; deps via deno.json). The existing publish.yml nonetheless runs npm ci / npm publish. Because there is no package.json, there is no repository field present for npm provenance to reference — npm requires a repository field that matches the GitHub repo for --provenance to succeed at publish time. This was not fixed here (additive provenance change only); flagging for owner attention.
  • Top-level permissions: contents: read already existed at file head; the job-level block is additive and scoped to the publish job.

🤖 Generated with Claude Code

@hyperpolymath

Copy link
Copy Markdown
Owner Author

⚠️ Holding this one. preference-injector has no package.json but publish.yml runs npm publish from root — so that npm publish is pre-existing non-functional (npm can't publish without a manifest), and --provenance is moot until that's resolved. Real question for the owner: is this repo meant to publish to npm at all (it's Deno/ReScript)? If yes → add a package.json with a repository field, then --provenance works. If no → drop the npm-publish job. Leaving this PR open as the flag rather than merging a no-op onto a broken publish.

@hyperpolymath
hyperpolymath merged commit 3c277ec into main Jun 25, 2026
11 of 27 checks passed
@hyperpolymath
hyperpolymath deleted the feat/package-provenance branch June 25, 2026 09:18
@sonarqubecloud

Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant